Skip to content

Make properties of *_Some_References deeply optional#65

Merged
OlivierCosti merged 1 commit intomasterfrom
deep_optional_references_properties
Apr 23, 2026
Merged

Make properties of *_Some_References deeply optional#65
OlivierCosti merged 1 commit intomasterfrom
deep_optional_references_properties

Conversation

@SH-Exact
Copy link
Copy Markdown
Contributor

This PR makes all properties of *_Some_References types, root level and nested properties, optional. This ensures convenience so that not all nested properties have to be set, but following is sufficient enough

wServices.article.some({
  include: {
    quantityConversions: {
      unitId: true
    }
  }
});

Before one had to set all unwanted properties to false like this

wServices.article.some({
  include: {
    quantityConversions: {
      createdUserId: false,
      lastEditedUserId: false,
      unitId: true
    }
  }
});

@SH-Exact SH-Exact self-assigned this Apr 20, 2026
@OlivierCosti OlivierCosti self-requested a review April 22, 2026 11:11
@SH-Exact SH-Exact force-pushed the deep_optional_references_properties branch from a8153f2 to ad92315 Compare April 23, 2026 06:22
@OlivierCosti OlivierCosti merged commit dba0498 into master Apr 23, 2026
8 checks passed
@OlivierCosti OlivierCosti deleted the deep_optional_references_properties branch April 23, 2026 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants